Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "Include more info if the gradle build fails" #22344

Merged
merged 1 commit into from
Dec 18, 2021
Merged

Revert "Include more info if the gradle build fails" #22344

merged 1 commit into from
Dec 18, 2021

Conversation

famod
Copy link
Member

@famod famod commented Dec 18, 2021

This reverts commit 2fac515. (from #20464)

The resulting archive exceeds 400 megs and it appears to me that we don't need all that anymore, but I might be wrong.
Alternatively, we could try to fine tune the pattern instead of merely going back to just test reports.

Draft until the decision has been made.

/cc @glefloch @stuartwdouglas @aloubyansky

@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Dec 18, 2021
@gsmet
Copy link
Member

gsmet commented Dec 18, 2021

Outch, I missed this change. This is very undesirable. If you want to do it, please do it in another artifact as the bot downloads these ones.

So I will merge and if you want these reports, please add another artifact.

@gsmet gsmet marked this pull request as ready for review December 18, 2021 14:06
@gsmet gsmet merged commit 8eef3ec into quarkusio:main Dec 18, 2021
@quarkus-bot quarkus-bot bot added this to the 2.7 - main milestone Dec 18, 2021
@famod famod deleted the revert-gradle-info branch December 18, 2021 14:59
@stuartwdouglas
Copy link
Member

I added this when we were having a lot of stability issues, I think reverting is fine.

@gsmet gsmet modified the milestones: 2.7 - main, 2.6.1.Final Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants